Skip to content

chore: use skills instead of rules#16

Open
strausr wants to merge 9 commits into
mainfrom
use-skills-instead-of-rules
Open

chore: use skills instead of rules#16
strausr wants to merge 9 commits into
mainfrom
use-skills-instead-of-rules

Conversation

@strausr

@strausr strausr commented May 21, 2026

Copy link
Copy Markdown
Collaborator

use skills instead of rules

carlevison
carlevison previously approved these changes May 26, 2026

@carlevison carlevison left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few suggestions.

Comment thread cli.js Outdated
Comment thread cli.js Outdated
Comment thread cli.js Outdated
Co-authored-by: carlevison <54800761+carlevison@users.noreply.github.com>
strausr and others added 5 commits May 26, 2026 08:38
Co-authored-by: carlevison <54800761+carlevison@users.noreply.github.com>
Co-authored-by: carlevison <54800761+carlevison@users.noreply.github.com>
@strausr strausr requested a review from carlevison June 8, 2026 18:25
Comment thread templates/AI_CONTEXT.md.template Outdated

## Transformation quick reference

URL structure: `https://res.cloudinary.com/{{CLOUD_NAME}}/image/upload/<transforms>/<public_id>`

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought the transformation quick reference may be a bit more detailed than this. But actually, neither of these guidelines are relevant to React unless it's for addTransformation.

@carlevison carlevison left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one suggestion, otherwise approved.

| Question type | Use |
|---------------|-----|
| Common patterns, imports, config, errors | This file |
| Transformation URL (common cases) | Quick reference below |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe don't use 'URL' here - could be just 'Transformations'.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants